Skip to content
This repository has been archived by the owner on Dec 7, 2021. It is now read-only.

Add better keyboard support #51

Merged
merged 1 commit into from
Sep 15, 2018
Merged

Add better keyboard support #51

merged 1 commit into from
Sep 15, 2018

Conversation

ericboehs
Copy link
Contributor

@ericboehs ericboehs commented Nov 13, 2017

After tabbing to a selectr element, pressing the typical keyboard activation keys (space, up/down) wouldn't open the selectr container. Pressing Enter wouldn't submit the form (or do anything).

After dismissing the dialog (e.g. via pressing enter), the tab focus was lost thus starting you back over at the top of the document (making using keyboard only navigation painful).

@ericboehs
Copy link
Contributor Author

ericboehs commented Nov 14, 2017

On second thought, I'm not sure I want Enter to activate the select as typically that's reserved for submitting the form. Space or the arrow keys are typically used to activate a select field.

I've updated the pull request.

@ericboehs ericboehs changed the title Fix Enter key and maintain activeElement after closing select Add better keyboard support Nov 14, 2017
After tabbing to a selectr element, pressing enter wouldn't activate the
selected state to display the selectr container.

After dismissing the dialog (e.g. via pressing enter), the tab focus was
lost thus starting you back over at the top of the document (making
using keyboard only navigation painful).
@Mobius1 Mobius1 merged commit eb6bc31 into Mobius1:master Sep 15, 2018
@ericboehs ericboehs deleted the enter-key branch September 17, 2018 17:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants